text: Be more selective when selecting on focus-in

We don't want to select on focus-in when the focus
comes from a child. The case where this does harm
is when you activate copy or paste actions from the
context menu. We close the menu before triggering the
action, and if that causes the text in the entry to
be selected, unexpected things happen, since the action
applies to the current selection.

Fixes: #2869

Revert "text: Avoid creating a PangoAttrList we don't need"

This reverts commit 67c26650280ba64ff1c0318be8d848e9563ddb21.

The splicing we do here has the important side-effect
of shifting the preedit attributes to the right position.
Without it, we end up always underlining the first chars
in the entry, regardless where the preedit happens.

text: Handle key events in the target phase

This makes sure that we do actual key input right
in the middle between all the capture and bubble
event controllers, and are not dependent on the
ordering of those controllers.

The bug that triggered this change was that the
shortcut for activation (Enter) was getting triggered
before the key input, causing Ctrl-Shift-u hex
to stop working, since it never received the enter
to commit the sequence.

widget: Add a debug message for consumed key events

Run the application with GTK_DEBUG=keybindings to
get some hints where key events get lost.

text: Give names to event controllers

It helps with debugging.

Merge branch 'wip/carlosg/sequence-state-fixups' into 'master'

Fixups to gesture sequence states

Closes #2895

See merge request GNOME/gtk!2154

gtktext: Don't accept (twice!) all press actions

The gesture should claim the sequence after triggering uncancellable
actions, like pasting, showing a menu or selecting words/lines. A
single first button press initiating a drag does not trigger
anything yet, so it should avoid claiming the sequence.

gtkswitch: Don't be eager in accepting the gesture

The gesture should be accepted whenever it triggers uncancellable
actions in the widget. This means it should be accepted if the
click does result in toggling the switch.

This leaves the pan gesture room to handle dragging the handle.

Fixes: https://gitlab.gnome.org/GNOME/gtk/-/issues/2895

Merge branch 'matthiasc/for-master' into 'master'

main: Avoid a warning

Closes #2894

See merge request GNOME/gtk!2153

docs: Some tweaks to the list widget overview

Merge branch 'wip/otte/bitset' into 'master'

Improve selection handling API for rubberbanding

See merge request GNOME/gtk!2086

main: Avoid a warning

transient-for relationships only exist between
windows, so check that both candidates are such.

Fixes: #2894

Merge branch 'tintou/dropdown-annotations' into 'master'

dropdown: Annotate the get_selected_item method

See merge request GNOME/gtk!2152

Merge branch 'wip/ricotz/annotations' into 'master'

gtk: Add some g-i annotations to GtkStringList

See merge request GNOME/gtk!2150

dropdown: Annotate the get_selected_item method

We need to specify the type as we already know that it is at least a GObject and in case of no selection, NULL is returned.

listbase: Make rubberbanding a threshold drag

listbase: Only compute the modifiers when releasing the rubberband

... and do the right things:

nothing:    selection = rubberband
ctrl:       selection = selection OR rubberband
shift:      selection = selection AND (NOT rubberband)
ctrl+shift: selection = selection XOR rubberband
            (not sure this one makes sense, but toggling is fun)

listbase: Compute rubberband region on-demand

Instead of storing the active items as we go, compute the affected items
whenever the rubberband changes and in particular when the rubberband
ends.
That way, the rubberband is guaranteed to select a rectangle even
after scrolling very far.

This is achieved by having a get_items_in_rect() vfunc that selects all
the items in the rubberbanded rectangle and returns them as a bitset.

listview: Allocate rubberband at end of size_allocate()

Otherwise the rubberband uses the wrong scroll offsets.

listbase: Flip autoscroll deltas if adjustments are flipped

Fixes autoscroll on RTL languages.

listbase: Allocate gridview items properly on RTL

We need to flip the items.

multiselection: Implement get_selection_in_range()

listbase: Allocate rubberband according to list coords

The rubberband is now handled on the list coordinate system.

When starting the rubberband, we track the item under the pointer and
follow it when it is moving.
This may lead to the rubberband start position changing position and
while this may be confusing, it alerts users to the fact that something
crazy is going on.

listbase: Move a common function from the children into GtkListBase

We want to use it for the rubberband later.

listbase: Don't do extra work

Scrolling causes a queue_resize() which will update the rubberband in
size_allocate() and queue a draw.

Fix indentation

gtk: Remove GtkPropertySelection

GtkMultiSelection is so much faster than this that it isn't needed.

gtk-demo: Port listview-colors demo to multiselection again

multiselection: Track items across resorts

In particular, track which items remain in ::items-changed
signal emissions.

But the main use case is sorting, which causes items-changed(0, n, n)
to be emitted.

Remove GtkSet

It's been superceded by GtkBitset.

selectionmodel: Remove select_callback() functions

gtk_selection_model_set_selection() takes care of those now.

listbase: Port rubberband to gtk_selection_model_set_selection()

selectionmodel: Add gtk_selection_model_set_selection()

Also port the testsuite.

selectionmodel: Replace query_range() with get_selection()

multiselection: Port to GtkBitset

testsuite: Add tests for GtkBitset

Add GtkBitset

gtk: Add some g-i annotations to GtkStringList

Add amalgamated roaring bitmaps source code

Taken from https://github.com/RoaringBitmap/CRoaring and fixed to
not spew warnings.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!2149

inspector: Adapt to dropdown api change

The suppported syntax for dropdowns in ui files
has changed. Get with the program.

docs: Fix a typo

gtk-demo: Fix a typo in the listbox demo

searchentry: Update the docs slightly

Update the docs to get closer to describing reality.

Drop unused includes

Don't include gtkfilechooserentry.h where it
isn't used.

stringfilter: Fix a typo

Update Turkish translation

Merge branch 'wip/carlosg/input-cleanups' into 'master'

Some input cleanups

Closes #2851

See merge request GNOME/gtk!2147

Merge branch 'scrolled-window-api' into 'master'

scrolledwindow: Don't take adjustments in new()

See merge request GNOME/gtk!2146

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!2145

gtktextview: Remove child textview items last

The function tries to go through every child, so it's convenient
to remove all internal ones first.

gtkwidget: Drop _gtk_widget_[gs]et_shadowed()

This API is meaningless now, with grab-notify being handled internally.

gtkwidget: Avoid use of shadowed flag

This is going away, look up whether there is a grab, and whether
the widget is within it, instead.

gtkwidget: Drop gtk_widget_device_is_shadowed()

A widget no longer needs updating on GTK grabs on itself, drop
this miscellaneous and no longer used API.

gtkwidget: Drop internal checks in grab-notify

There's no need to check if the widget is already shadowed, we
implicitly have that from the signal argument.

gtkmain: Make grab-notify notifications more targeted

We only want to send grab-notify to widgets that might have been
interacting with devices via events. Instead of going through all
widgets in all toplevels, we have the window/pointer focus information,
so we can just traverse the widget stacks for every involved foci.

gtkwidget: Drop ::grab-notify signal and vfunc

We don't expect widgets to react directly to these anymore.
Internally we still reset the controllers.

gtkprintoperation-win32: Don't track grab-notify

Further grabs could presumably only come from events handled within
the dialog itself. It seems overeager to do this.

gtkvideo: Track child focus to determine whether controls should autohide

Move away from grab_notify, and use the set_child_focus() vfunc to track
child widgets being set the keyboard focus. This is not 1:1, but seems
good enough at the moment.

gtkbutton: Drop grab_notify handler

The click gesture already has a ::cancel handler, and does almost
everything it should. Move the last bits and remove the grab_notify
handler.

gtkspinbutton: Drop grab_notify handler

The press gesture on the buttons already has a ::cancel handler,
this is redundant.

gtknotebook: Move away from grab_notify

Connect to ::cancel on the relevant gesture instead.

testgtk: Update list of cursor names

Use the standard names, since that is what we want
everybody else to use too.

Install gtkstringlist.h

entrycompletion: Various fixes

Fix various issues that prevented inline completion
and inline selection from working reliably. We were
passing byte counts to gtk_editable_select_region in
one place, but that function expects char counts.
We were listening for GtkEditable::insert-text on
the GtkText widget, but that does not emit those signals,
so listen for GtkEntryBuffer::inserted-text instead.
Finally, we were not clearing the stored completion_prefix
enough, leading to situations where the stored prefix
does not match the text in the entry anymore.

gtk-demo: Enable inline completion

Enable inline completion in the entry completion
demo, to have some code exercising this feature.
It is broken.

gtk-demo: Improve the entry completion demo

Add more strings.

scrolledwindow: Don't take adjustments in new()

In 99.9% of all cases, these are just NULL, NULL.
So just do away with these arguments, people can
use the setters for the rare cases where they want
the scrolled window to use a different adjustment.

gtkwindowhandle: Drop has-grab checks

While it's worth thinking about bringing the "windows can be dragged
with open popovers" behavior back, this does not kick in anymore, nor
should be the way to handle this given all the autoclose surface
semantic changes.

gdk: Drop gdk_device_get_last_event_surface()

This kind of transient state sets the expectative that events update
devices, while it's more accurate to say that devices generate events.
It does not make to expose this function anymore.

gtkwidget: Avoid poking surfaces and events in gtk_widget_list_devices()

This got stuck in ancient times when widgets were windowed, so the devices
in a window to know the devices in that widget would pan out. We do only
want here the devices that are inside the widget, not spread over the
surface, so rewrite this helper function to poke the toplevel foci, and
look they are contained inside the widget.

gtkwidget: Drop _gtk_widget_get_device_surface()

A widget cannot have multiple surfaces anymore, nor needs to
fetch the last event surface as it ought to handle its own.

gtkmain: Drop unused struct field

Grabs are, ironically, not about devices anymore. This struct field
is never set.

gtkwidget: Remove dead code

This variable got unused over time.

gdk: Depend less on gdk_event_get_device()

For the most part, we are interested in seats here.

gdk: Add gdk_event_get_seat()

It currently fetches the seat from the event device, will be changed
in future commits.

gdk: Drop gdk_seat_get_logical_pointers()

Events come from hardware devices and are handled by controllers,
there's no need to use logical pointers, nor to peek them. Drop this
unused API.

gdk: Drop GdkGrabOwnership

We no longer expose such low level tweaks, this is essentially
unused.

gdk: Drop supports_multidevice API

Crossing events are now detached from widget state, all tricky consequences
from getting multiple crossing events are now somewhat moot. Resort to sending
all generated crossing events, and drop this barely (ever?) used API.

gdk: Drop GDK_SOURCE_ERASER

All tools come from devices with GDK_SOURCE_PEN.

gtkmain: Look up transient-for hierarchies to determine modality

Windows that are not modal, but are transient-for a modal window should
still be able to receive and handle events. Inspect the window hierarchy
in those cases, so these windows are handled just like widgets within
the modal dialog.

Fixes: https://gitlab.gnome.org/GNOME/gtk/-/issues/2851
Related: https://gitlab.gnome.org/GNOME/gtk/-/issues/2850

gtkwidget: Cancel all gestures in other groups

When a gesture (group) claims a sequence, all other gesture groups
in the same widget should get cancelled. Not just previously claimed
ones, that shouldn't happen actually.

Merge branch 'stringlist' into 'master'

add a public stringlist api

See merge request GNOME/gtk!2134

Add GtkStringList as public api

This is a list model holding strings, initialized
from a char **. String lists are buildable as well,
and that replaces the buildable support in GktDropDowns.

Merge branch 'wip/jimmac/notebook-legibility' into 'master'

Adwaita: notebook legibility fixes

Closes #2652

See merge request GNOME/gtk!2142

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!2141

Adwaita: notebook legibility fixes

- make inactive tabs more legible

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/2652

Add a few missing includes

These sources are using GtkListStore apis,
but were replying on indirect includes to
get the header. Make this explicit, to prepare
for GtkEntryCompletion losing its tree view
dependencies.

entry completion: Drop the Private struct

This is in preparation for porting GtkEntryCompletion
to use GtkListView.

Merge branch 'wip/fencing-without-context-is-discouraged' into 'master'

x11: Ensure bound context is compatible with sync fence

See merge request GNOME/gtk!2139

x11: Ensure bound context is compatible with sync fence

Commit a0f6ff101e781aaf0a931be6e22f2885f4f69e04 made sure that a
context was bound before calling glClientWaitSync, but it doesn't
check that the context shares objects with the context that created
the fence.

This commit does a little more validation before deciding the current
context is good enough.

Merge branch 'wip/fencing-without-context-is-discouraged' into 'master'

x11: ensure some context is bound before calling glClientWaitSync

See merge request GNOME/gtk!2138

Merge branch 'drop-entry-completion-actions' into 'master'

entrycompletion: Drop action support

See merge request GNOME/gtk!2136

Merge branch 'matthiasc/for-master' into 'master'

Add a basic clipboard test

See merge request GNOME/gtk!2137

x11: ensure some context is bound before calling glClientWaitSync

Since commit 972134abe48a4c9c7b6ad41b0723f30f4e7ae16b we now call
glClientWaitSync for the vendor nvidia driver, to know when a frame
is ready for the compositor to process.

glClientWaitSync can be called regardless of which context is currently
bound, but if no context is bound at all, it returns 0 without
doing anything.

This commit checks for that edge case, and ensures a context gets
made current in the event no context is already current, before calling
glClientWaitSync.

Add a basic clipboard test

This is not reproducing the failure I'm after, unfortunately.

entrycompletion: Drop action support

This api has not really been kept up with current
user experiences in popups, and we're better off
just dropping it and letting people do their own
popups if they need custom UI.

Merge branch 'keyval-to-uni-fix' into 'master'

gdk: Make gdk_keyval_to_unicode platform-independent

See merge request GNOME/gtk!2132

Merge branch 'fix-entry-completion' into 'master'

Fix entry completion

See merge request GNOME/gtk!2135

gdk: Clarify the docs of gdk_keyval_to_unicode

Explicitly state that the conversion does not
take locale into account, and point out GDK_KEY_KP_Decimal
as a candidate for special-casing.